Skip to content

Enable comparator on nRF54L20 #2781

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

nordic-segl
Copy link
Contributor

@nordic-segl nordic-segl commented Apr 17, 2025

Extend comparator drivers with analogue input to GPIO pin mapping.
In comparator test, use other analogue inputs.

manifest-pr-skip

nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Apr 17, 2025
@nordic-segl nordic-segl force-pushed the NRFX-7312_Enable-comparator-on-54L20 branch from 775da1e to b5faf5c Compare April 17, 2025 06:10
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Apr 17, 2025
…F54L20

Add mapping from analogue input channel to GPIO number on nRF54L20.
Make changes to both comparator and low power comparator.

Upstream PR #: 88745

Signed-off-by: Sebastian Głąb <[email protected]>
Test uses AIN4 and AIN5.
Unfortunately, these inputs on nRF54L20 are used
by serial port console.

When test is compiled for nRF54L20, use other anlogue inputs.

Upstream PR #: 88745

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-7312_Enable-comparator-on-54L20 branch from b5faf5c to 954eaf2 Compare April 18, 2025 12:19
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
60.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@@ -15,20 +15,30 @@ static const struct gpio_dt_spec test_pin_2 = GPIO_DT_SPEC_GET(DT_PATH(zephyr_us


struct comp_nrf_comp_se_config comp_se_config = {
#if defined(CONFIG_BOARD_NRF54LM20APDK)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no such board at upstream, maybe feature flag?

nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Apr 22, 2025
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Apr 23, 2025
@nordic-segl
Copy link
Contributor Author

Implemented in a different way.

@nordic-segl nordic-segl deleted the NRFX-7312_Enable-comparator-on-54L20 branch May 14, 2025 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants